Skip to content

Conversation

@jonathan-buttner
Copy link
Contributor

This PR addresses the feedback item from the original PR: #127939

This PR renames a test file and flattens the request map. Instead of having {... "request": {"content": <content string> } } it changes it to {... "request": <content string> }

Comments:
#127939 (comment)
#127939 (comment)

@jonathan-buttner jonathan-buttner added >non-issue :ml Machine learning Team:ML Meta label for the ML team auto-backport Automatically create backport pull requests when merged v8.19.0 v9.1.0 labels May 30, 2025
@jonathan-buttner jonathan-buttner marked this pull request as ready for review June 2, 2025 13:01
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/ml-core (Team:ML)

Copy link
Member

@davidkyle davidkyle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jonathan-buttner jonathan-buttner merged commit 1199b10 into elastic:main Jun 3, 2025
18 checks passed
@jonathan-buttner jonathan-buttner deleted the cs-flatten-request-content branch June 3, 2025 17:37
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
8.19

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged :ml Machine learning >non-issue Team:ML Meta label for the ML team v8.19.0 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants